Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add weather app to demos #2597

Merged
merged 3 commits into from
Oct 24, 2024
Merged

Add weather app to demos #2597

merged 3 commits into from
Oct 24, 2024

Conversation

AJBrownson
Copy link
Contributor

@AJBrownson AJBrownson commented Oct 15, 2024

Description

  • Adding a new demo application in the /demos folder
  • Addressing hacktoberfest issue: 🤩 Create Weather App (Hacktoberfest Demo) #2566
  • Elemental is a weather app that gets the current user's location and shows the weather information for that location. It also
    allows users to search for weather information by city name. It uses the OpenWeather API to retrieve weather data and the
    Tolgee API to provide translations.

Key Features:

  • Text Translation
  • Weather Information
  • Search by City Name
  • Responsive Design

Screenshots

Weather App Screenshot

Demo

View the demo live here

Article

Read the DEV article here

Checklist

  • [✓] I have read the Contribution Guide
  • [✓] If the PR changes or adds functionality, I have updated the relevant documentation
  • [✓] Translate all labels in the App with Tolgee
  • [✓] Store the project into /demos folder
  • [✓] Add a screenshot or video demo to the PR
  • [✓] Write a short article describing your demo app on DEV.to or your blog
  • [✓] Please show your support and give a ⭐ to the Tolgee repository

Copy link
Contributor

@JanCizmar JanCizmar left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks a lot for the demo. 🚀

Just 2 things

  • Please send a link to the demo on Vercel
  • Please create the dev.to article


Add the following code to your main.tsx file:

```typescript
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Why do I need to do so? It's already there in the main.tsx?

Copy link
Contributor Author

@AJBrownson AJBrownson Oct 21, 2024

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I put that in the README to guide future users
I've removed it

```typescript
const tolgee = Tolgee()
.use(DevTools())
.use(BackendFetch()) // add this particular line
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This line is already there.

```bash
npx tolgee login <your-tolgee-api-key>

npx tolgee --project-id <your-project-id-number> pull --path public/i18n
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The files are already there, so you can host it on Vercel as it is. You have to run this only, when you need to update translations.

Copy link
Contributor Author

@AJBrownson AJBrownson Oct 21, 2024

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I was following this tutorial I found online
Removing the line now

@AJBrownson
Copy link
Contributor Author

@JanCizmar
I have added the vercel demo link and the dev.to article link also

@AJBrownson
Copy link
Contributor Author

@JanCizmar
I have made the correct commits
Waiting for your review

@JanCizmar JanCizmar self-requested a review October 24, 2024 13:51
Copy link
Contributor

@JanCizmar JanCizmar left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks fine! Thanks a lot! 🚀

@JanCizmar JanCizmar merged commit cd4bd6c into tolgee:main Oct 24, 2024
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants